Skip to content

RI-661 Implement RPC-Designate MNAIO job#3484

Open
Frank-ZhangXin wants to merge 1 commit into
rcbops:rockyfrom
Frank-ZhangXin:add-designate-mnaio-check
Open

RI-661 Implement RPC-Designate MNAIO job#3484
Frank-ZhangXin wants to merge 1 commit into
rcbops:rockyfrom
Frank-ZhangXin:add-designate-mnaio-check

Conversation

@Frank-ZhangXin

@Frank-ZhangXin Frank-ZhangXin commented Feb 18, 2019

Copy link
Copy Markdown
Contributor

Implement rpc-designate job as one 'action', which could be used
later in rpc-gating script. This way is re-using rpc-artifacts's
mnaio vm image and then apply rpc-designate testing upon it.

Issue: RI-661

Implement rpc-designate job as one 'action', which could be used
later in rpc-gating script. This way is re-using rpc-artifacts's
mnaio vm image and then apply rpc-designate testing upon it.
@rpc-jenkins-svc

Copy link
Copy Markdown

Thank you for submitting a pull request to this project!

Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.

When the gate jobs have completed successfully, this pull request will get
merged automatically.

@Frank-ZhangXin

Copy link
Copy Markdown
Contributor Author

Got unexpected exception:

MNAIO RPC-O deploy completed...

  • [[ designate == \d\e\p\l\o\y ]]
  • [[ designate == system* ]]
  • [[ designate == \s\d\q\c ]]
  • [[ designate == \e\l\k ]]
  • [[ designate == \d\e\s\i\g\n\a\t\e ]]
    +++ dirname ./gating/check/run
    ++ readlink -f ./gating/check/run_designate_tests.sh
  • bash -c /opt/rpc-openstack/gating/check/run_designate_tests.sh
    bash: /opt/rpc-openstack/gating/check/run_designate_tests.sh: Permission denied

@jpward1981 jpward1981 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, fix the permissions issue and we should be good to merge

@odyssey4me

Copy link
Copy Markdown
Contributor

Until #3509 merges, the syntax/lint tests will fail.

@odyssey4me

Copy link
Copy Markdown
Contributor

recheck_container_ansible-lint_tox-test
recheck_container_ansible-syntax_tox-test
recheck_xenial_no_artifacts_swift_deploy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants